-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Print LmodError when loading GCCcore-12.2.0-based modules on zen4
#841
Print LmodError when loading GCCcore-12.2.0-based modules on zen4
#841
Conversation
…dError when they are loaded
Instance
|
Instance
|
Instance
|
…e of the step-hooks, so we can unset it at the end
Ok, this PR is more or less ready, but we should create a known issues page on the zen4 tree missing |
Ok, I'll need EESSI/docs#357 to be merged first. Then, I'll put a link to that part of the docs in the LmodError. |
To test this PR:
I think all of these look as expected and intended. With that, I think this PR is ready for a final review (and hopefully merge :)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR merged! Moved |
1 similar comment
PR merged! Moved |
bot: build repo:eessi.io-2023.06-software arch:aarch64/neoverse_n1 |
Updates by the bot instance
|
Updates by the bot instance
|
Unable to download or merge changes between the source branch and the destination branch. |
We can take care of the ingest via #608 which is now ready to deploy & merge? |
Implements the idea from https://gitlab.com/eessi/support/-/issues/37#note_2159031831
But, not currently working, because the first module that gets installed that uses GCCcore-12.2.0 as dependency will try to load it (even with
--module-only
), which then fails:maybe we can make that pre-prepare hook do nothing. Or skip the prepare phase. NOt sure...